Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(concerto): fix bugs when id field has regex #630

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

DianaLease
Copy link
Member

@DianaLease DianaLease commented Mar 24, 2023

Closes #629

Changes

  • Throws an error if an id is provided to factory. newResource() that does not match the regex of the identifying field
  • Fixes cli generate to generate an id that matches the regex if there is regex on the identifying field

Flags

  • Had to update the tests using "CAR_123" since that id did not match the regex for VIN and now caused an error to throw.

Screenshots or Video

Related Issues

Author Checklist

  • Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
  • Vital features and changes captured in unit and/or integration tests
  • Commits messages follow AP format
  • Extend the documentation, if necessary
  • Merging to main from fork:branchname

@DianaLease DianaLease requested review from dselman, mttrbrts and a team March 24, 2023 00:24
@DianaLease DianaLease changed the title fix(concerto): fix bugs then id field has regex fix(concerto): fix bugs when id field has regex Mar 24, 2023
@DianaLease DianaLease force-pushed the dlease/fix-id-with-regex-bug branch from 5b60a62 to e1506b8 Compare March 24, 2023 00:25
@DianaLease DianaLease force-pushed the dlease/fix-id-with-regex-bug branch from e1506b8 to 7c6f05d Compare March 24, 2023 00:25
@mttrbrts mttrbrts merged commit 77d216f into main Mar 24, 2023
@mttrbrts mttrbrts deleted the dlease/fix-id-with-regex-bug branch March 24, 2023 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generating a sample breaks when regex used on identifier
2 participants